Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bootstrap testing #47

Merged
merged 1 commit into from
Oct 31, 2024
Merged

Bootstrap testing #47

merged 1 commit into from
Oct 31, 2024

Conversation

lulu-berlin
Copy link
Member

@lulu-berlin lulu-berlin commented Aug 19, 2024

This pull request addresses #44 by bootstrapping testing and implementing one unit test.

Changes:

  • Add a unit test for the baseline configuration
  • All mocks are extracted into conftest.py
  • Add fixture files for hardware-configuration.nix and kbd-model-map
  • Add a nix flake for running and developing the tests
  • Add instructions to README.md
  • Configure github actions to run tests on all pull requests

@vlinkz @ElvishJerricco

- Add a unit test for the baseline configuration
- All mocks are extracted into `conftest.py`
- Add fixture files for `hardware-configuration.nix` and `kbd-model-map`
- Add a nix flake for running and developing the tests
- Add instructions to README.md
- Configure github actions to run tests on all pull requests
@lulu-berlin
Copy link
Member Author

You can see on my fork that the github actions job works fine (and only takes 25 seconds): https://github.com/lulu-berlin/calamares-nixos-extensions/actions/runs/10458771458/job/28961314243

@lulu-berlin lulu-berlin mentioned this pull request Aug 19, 2024
@lulu-berlin
Copy link
Member Author

@vlinkz, it occurred to me that it might be unclear what this PR does exactly or how it does it. Would it help if I added documentation explaining how it works and what it tests?

@vlinkz vlinkz merged commit 2fa5064 into NixOS:calamares Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants